London | 26-ITP-Sep| Mandip Sanger | Sprint 2 | Coursework - #1469

Good start, I have some comments about this one though

There are still a lot of issues to fix before this PR can be closed. I am going to help you out a bit, but you need to pay close attention to the questions and feedback you are getting.

1. 2-mandatory-errors/0.js has not been started. Put // at the start of lines 1 and 2.

2. 1-percentage-change.js line 5 is missing a comma. Change replaceAll("," "") to replaceAll(",", "") and the file will run.

3. 1-percentage-change.js c) asks which lines are reassignments. The answer is lines 4 and 5. What you have under c) now is the answer to b), so move it to b).

4. 2-time-format.js line 32 says the code works for all values. It does not. It works for non-negative whole numbers, which is what you wrote in your reply on this PR. Put that in the file.

5. 2-mandatory-errors/1.js. The fix is right, but the section also asks you to explain the error, and you have not done that anywhere yet. Run the original and node prints TypeError: Assignment to constant variable. It happens because age is a const, and a const cannot be given a new value. Write that down, either in the file as a comment or as a reply here. Whichever you prefer.

6. 2-mandatory-errors/4.js. The renaming is right, but your comments explain how to convert 12 hour time into 24 hour time, which is a different question. The error is SyntaxError: Invalid or unexpected token, because a variable name cannot start with a digit. Explain that instead, again either in the file or as a reply here.

Your explanation in 4-random.js is well done.

Run each file once after you change it. Add the Needs Review label when you have pushed.

Points 1, 2, 3 and 5 are done, and every file runs now. Thank you.

Two things are still open:

1. 2-time-format.js lines 32 and 33 still say the code works for all values. This is the third time we have asked about f). Delete lines 32 and 33. In their place, copy the two sentences from your reply on 17th Sep:

   the code will not work correctly for all values of movieLength.

   It works correctly for non-negative whole numbers (integers) representing seconds.

2. 2-mandatory-errors/4.js: see my comment on line 4.

When you push these, I will mark this PR Complete. Add the Needs Review label again when you have pushed.